Adopt the console Python SDK for control-plane tools - #91
Merged
Conversation
Greptile SummaryThe PR replaces the server SDK with
Confidence Score: 5/5The PR appears safe to merge, with no concrete changed-code defect identified. The catalog filtering, target-context enforcement, SDK migration, and transport-specific registration paths are internally consistent, and no reachable failure remained after reviewing their call flows and coverage. Important Files Changed
Reviews (1): Last reviewed commit: "(feat): adopt console Python SDK" | Re-trigger Greptile |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
appwritedependency with the publishedappwrite-console0.2.1 SDK.model_typeparameter from generated MCP schemas.Catalog policy
The authentication boundary lives in
catalog_policy.py:The public MCP surface remains unchanged at up to four operator tools.
Live verification
The new OAuth integration suite exercises one safe read through the MCP registration/execution path for every non-billable added service. Against staging, all 14 probes passed, including regions, organizations, domains, projects, notifications, DocumentsDB, migrations, dedicated databases, usage, VCS, VectorsDB, and WAF.
Text embeddings are available in both applicable catalogs, but their live probe requires
APPWRITE_TEST_BILLABLE_EMBEDDINGS=1to avoid unexpected charges.Test plan
uv run --group dev ruff check src testsuv run --group dev black --check src testsuv run --group dev pyrightuv run python -m unittest discover -s tests/unit -v(200 tests)uv builddocker build -t appwrite-mcp:ci .(local Docker/OrbStack daemon unavailable; CI will run this)